-
Notifications
You must be signed in to change notification settings - Fork 244
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dycore semi-implicit solver update #1720
Dycore semi-implicit solver update #1720
Conversation
@XiaqiongZhou-NOAA sounds like fv3 dependency PR was tested on wcoss2. If test result (log file) is attached to the pr, it will speed up the commit process. I mean if available. Either way, we will use jenkins-ci as pre-test. |
I have merged latest UFS to my branch and will rerun the regression tests to create a new baseline on wcoss2. |
@XiaqiongZhou-NOAA hold a bit. We are working on #1509. You may try to re-run after #1509 is merged, We can help to re-run on Cheyenne. |
OK, but I am afraid I do not have access to Cheyenne.
…On Wed, Apr 26, 2023 at 8:55 AM JONG KIM ***@***.***> wrote:
@XiaqiongZhou-NOAA <https://github.com/XiaqiongZhou-NOAA> hold a bit. We
are working on #1509
<#1509>. You may
try to re-run after #1509
<#1509> is merged,
We can help to re-run on Cheyenne.
—
Reply to this email directly, view it on GitHub
<#1720 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALQE7UQVWG4TXTK2DDO5TQ3XDELMHANCNFSM6AAAAAAXJTLFOQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Sure! We still have time. We will help to test on Cheyenne while jenkin-ci runs. Hera scratch1 disk quota issue this morning. I am trying to clean up a bit. I will keep you posted. |
@XiaqiongZhou-NOAA We can start working on this pr. #1509 was merged. Can you sync up? |
Synced |
"Test 008 compile FAIL" in cheyenne.intel.log. Any issues there? |
cpld_control_p8_faster was turned off on cheyenne. I think we should turn off the 008 build on cheyenne as well. |
@jkbk2004 cpld_control_nowave_noaero_p8 failed the ORT debug test, when running jenkins-ci against this PR. Everything else passed. I'll look into the failure. |
@zach1221 we need a decision about jenkins-ci cpld_control_nowave_noaero_p8. Can you give a try to run ort cpld_control_nowave_noaero_p8 on here or hera? |
@jkbk2004 Yes, I'll test again on Hera now. |
@jkbk2004 cpld_control_nowave_noaero_p8 passed ORTs using both intel/gnu compilers on Hera. |
all test are done. we can start merging process. thanks @zach1221 for following with ORT. |
@XiaqiongZhou-NOAA fv3 pr was merged. Can you update hash and revert change in gitmodules? correct hash is NOAA-EMC/fv3atm@e0d95b1 |
Description
To bring semi-implicit solver update in dynamics to UFS. A new baseline should be created since it will change the results.
Input data additions/changes
Anticipated changes to regression tests:
all regression tests.
Subcomponents involved:
Combined with PR's (If Applicable):
Commit Queue Checklist:
Linked PR's and Issues:
Testing Day Checklist:
Testing Log (for CM's):